Skip to content

Added API Token object and logging in as a different user#2

Open
jessevz wants to merge 7 commits intomainfrom
test-improvements
Open

Added API Token object and logging in as a different user#2
jessevz wants to merge 7 commits intomainfrom
test-improvements

Conversation

@jessevz
Copy link
Copy Markdown

@jessevz jessevz commented Apr 28, 2026

No description provided.

@jessevz jessevz marked this pull request as ready for review April 29, 2026 11:46
@jessevz jessevz requested review from Copilot and s3inlc April 29, 2026 11:46
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support for interacting with Hashtopolis APIv2 using explicit credentials (for “log in as a different user”) and introduces an ApiToken API object.

Changes:

  • Added HashtopolisConfig.with_credentials() to construct configs without reading a config file.
  • Threaded optional auth credentials through QuerySet → connector filter() calls.
  • Added ApiToken model and exported it from the package.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
hashtopolis/hashtopolis.py Adds credential-based config construction, optional auth handling for filtering, QuerySet.authenticate(), and the new ApiToken model.
hashtopolis/__init__.py Exports ApiToken and adds Helper to the base import list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread hashtopolis/hashtopolis.py
Comment thread hashtopolis/hashtopolis.py
Comment thread hashtopolis/hashtopolis.py
Comment thread hashtopolis/__init__.py
Comment thread hashtopolis/__init__.py
@jessevz jessevz requested a review from s3inlc May 6, 2026 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants